coverage: 97.695%. remained the same — redb_3 into master

would it make sense to just jump ahead to redb 4.1 ? it looks like it has a bunch of bug fixes:

https://github.com/cberner/redb/releases/tag/v4.1.0

🤖 Code Review by Antigravity

Pull Request: #28 — build: bump redb version and MSRV
Review Status: 🟢 Approved with Suggestion

📝 Summary of Review

This PR updates bdk_redb to depend on redb v4.1.0, bumps the library version to 0.3.0 due to public API breaking changes (pub use redb;), and elevates the Minimum Supported Rust Version (MSRV) from 1.85.0 to 1.89.0 (required by redb 4.1.0).

✅ Verification Checklist

• Compilation: cargo check --all-targets --all-features passed without errors.
• Unit & Integration Tests:
  • cargo test --all-features passed (25 unit tests, 1 doctest).
  • cargo test --no-default-features passed (22 unit tests, 1 doctest).
• Linter: cargo clippy --all-targets --all-features passed with zero warnings.
• Documentation: cargo doc --all-features built cleanly without warnings.
• SemVer Rules: Version bumped to 0.3.0 appropriately reflecting redb major version bump.

🔍 Detailed File-by-File Review

1. Cargo.toml

• Changes: Bumps redb to "4", rust-version to "1.89.0", and bdk_redb crate version to "0.3.0".
• Feedback: ✅ Looks good. Correctly reflects breaking change rules and minimum toolchain requirement.

2. src/lib.rs

• Changes:
  • Adds ReadableDatabase trait to imports from redb.
  • Updates crate-level documentation example (bdk_redb = "0.3.0", adds Database and Store imports, removes deprecated .extract_keys()).
• Feedback: ✅ Necessary for trait method resolution in redb 4.x and keeps documentation code snippet working and runnable.

3. .github/workflows/cont_integration.yml, clippy.toml, rust-version

• Changes: MSRV updated to 1.89.0 across CI matrix, clippy configuration, and rust-version.
• Feedback: ✅ Good consistency across build & CI files.

⚠️ Suggested Action Item

Reviewed by Antigravity AI Coding Assistant